-
-
Notifications
You must be signed in to change notification settings - Fork 785
[17.0][OU-ADD] mass_mailing: Migration to 17.0 #4671
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
/ocabot migration mass_mailing |
carlos-lopez-tecnativa
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TT49916
| NEW mailing.contact: mass_mailing.mass_mail_contact_0 (noupdate) | ||
| DEL mailing.contact: mass_mailing.mass_mailing_contact_0 (noupdate) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I consider that this ID XML needs to be renamed instead of "Nothing to do" to prevent data loss.
mass_mailing.mass_mailing_contact_0 renamed to mass_mailing.mass_mail_contact_0
| ---Fields in module 'mass_mailing'--- | ||
| mass_mailing / mail.blacklist / opt_out_reason_id (many2one) : NEW relation: mailing.subscription.optout | ||
| mass_mailing / mailing.contact / _order : _order is now 'name ASC, id DESC' ('email') | ||
| mass_mailing / mailing.contact / list_ids (many2many) : table is now 'mailing_subscription' ('mailing_contact_list_rel') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs a migration script to rename the table from mailing_contact_list_rel to mailing_subscription
V16
https://github.com/odoo/odoo/blob/733b1474e75b32c3a467b7c3f0a83c75467d4e50/addons/mass_mailing/models/mailing_contact.py#L77C26-L77C50
V17
https://github.com/odoo/odoo/blob/f7eccbacae815b87b91abf923e19a4b6a97e5c40/addons/mass_mailing/models/mailing_contact.py#L37
| mass_mailing / mailing.contact.subscription / contact_id (many2one) : DEL relation: mailing.contact, required | ||
| mass_mailing / mailing.contact.subscription / list_id (many2one) : DEL relation: mailing.list, required | ||
| mass_mailing / mailing.contact.subscription / opt_out (boolean) : DEL | ||
| mass_mailing / mailing.contact.subscription / unsubscription_date (datetime): DEL | ||
| # NOTHING TO DO: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I interpret that Odoo renamed models from mailing.contact.subscription to mailing.subscription. So, we must add the respective migration script to fill the new table from the old table, or rename the tables (if possible) with the goal of preserving data.
|
Superseded by #4785 |
No description provided.